NUTCH-3130 Address deprecated API usage across Nutch codebase and build#869
NUTCH-3130 Address deprecated API usage across Nutch codebase and build#869lewismc wants to merge 17 commits intoapache:masterfrom
Conversation
|
|
||
| @Override | ||
| protected void finalize() throws Throwable { | ||
| shutDown(); |
There was a problem hiding this comment.
I'm not sure we can simply remove the call to shutdown. I need to further investigate options and confirm.
There was a problem hiding this comment.
@sebastian-nagel the smoke tests passes https://ci-builds.apache.org/job/Nutch/job/Nutch-Smoke-Test-Single-Node-Hadoop-Cluster/49/
I'm honestly not sure how to test this further... the same goes for similar removals in this PR.
|
|
||
| @Override | ||
| protected void finalize() { | ||
| /** |
There was a problem hiding this comment.
I'm also not sure on this right now. I need to do some more research.
|
I did want to also note, there are 15 instances of the @deprecated annotation across 9 files in the codebase: I'm not sure whether is the correct time to be removing these methods but if we decide to keep them we should definitely augment the Javadoc to detail the replacement API usage. |
sebastian-nagel
left a comment
sebastian-nagel
left a comment
There was a problem hiding this comment.
Great! Thanks, @lewismc!
Some work still to be done. Let me know whether I shall take over.
|
|
||
| @Override | ||
| protected void finalize() throws Throwable { | ||
| shutDown(); |
|
I didn't see you had responded here @sebastian-nagel . I will revisit this PR, update it and perform some more testing. |
|
I'm currently smoke testing the updated PR https://ci-builds.apache.org/job/Nutch/job/Nutch-Smoke-Test-Single-Node-Hadoop-Cluster/38 |
|
2 participants
Proposed fix for https://issues.apache.org/jira/browse/NUTCH-3130
This is a fairly straightforward PR. Some grunt work involved...
I took a bit of time to investigate issues with and options for removing instances of
@override finalize()... this may need more investigation and certainly some testing to make sure this PR doesn't break any behavior.